Essentially revert the skytraq part of PR#286.

Merge branch 'master' into atoi

Merge pull request #284 from gpsbabel/cl-cleanups

CodeLion-inspired and manual cleanups

add support for multiple urls for route headers. (#276)

* add support for multiple urls for route headers.

add support in gpx for gpx/rte/link, gpx/rte/url, gpx/rte/urlname.
add support in gpx for gpx/trk/link, gpx/trk/url, gpx/trk/urlname.

Note that the gpx writer can violate the schema when fprint_xml_chain
is used to echo unrecognized elements from input to output.
This is because the gpx 1.0/1.1 schema requires an xsd:sequence of
elements, i.e. the order of the elements is specified, and any
elements output by fprint_xml_chain won't necessarily be in
the correct order with other elements that are specifically
handled in the gpx writer.

By processing additional gpx elements as listed above on read we
prevent them from causing schema violations on write.  However,
other unrecognized elements can still cause schema violations.

* make common link code common in gpx reader.

Don't #include debug

A near rewrite of the reader of the stmsdf reader to more effectively parse fields as QDate,Times,Floats, etc. Still not very readable, but should be more bulletproof for bad input.

Don't name params in decls in itracku

Sign extension fix in EasyGPS

Remove xstrrstr and BASE_STRUCT from defs.h

Remove write-only variable in compegps.

Drop some unused method protos from CET.

XmlStreamWriter: make single arg methods explicit. Drop unused methods.

Removing scraggling bits of xhtmlent from build files.

Remove (now) unused xhtmlent table.

Make types in ggv_bin more safe for 32/64 systems.

Default construct to eliminate warning in geojson

Build fix for Cmake/CLion on Mac. Correct handling of frameworks. They're like
libs, but not...

add cppcheck target to GPSBabel.pro. (#280)

* add cppcheck target to GPSBabel.pro.

fix a few cppcheck errors.

* fix a few cppcheck items.

strptime fix is from upstream.

Wrap (use) of variables in #defines.

Warning fix in raymarine

Warning fix in locus

Fix warning in maggeo

Warning fixes in jeeps: move use closer to initialization so that the
test of device is effective, don't set locals before return.

Tame string formatting warning in libusb.

Remove unneeded const qualifier on pass-by-value types.

Replace size() == 0 with empty() when more readable.

Const-ipate jeeps function signature. (Yeah, coz THAT makes that API beautiful...)

Automated cleanups for passing by reference and fixing pointer ownership on calls.

Modernize loop iterators to walk arrays and containes instead of independently
maintained constants.

Modernize C style explicit (void) in decls to (), eliminate return
at end of bunction bodies.

Modernize bool use of true/false.

Make prototype arg names match function arg names.

Pick up a few old NULL->nullptr conversions that are stuck inside __APPLE that
tsteven4 dodged around since he couldn't confirm.

For single argument ctors, mark type explicit.

If there's only one argument in a ctor, mark it explicit to make it less
easy to fall into implicit conversions.

Diff and Terrain in geocaches are integers internally.

generate html coverage reports. (#281)

* generate html coverage reports.

* tweak coverage recipe.

remove obsolete msvc workarounds. (#278)

msvc 2012 and earlier are not supported.
They didn't support c++14 anyway which we currently require.

use QThread::usleep instead of our own function. (#277)

Our test suite doesn't test sleep, but the nmea pause option
can be used on write to verify functionality interactively.

obsolete DEBUG_MEM (#279)

simplify conversion from utm. (#275)

* simplify conversion from utm.

add test cases for Norway and Svalbard exceptions.

* Correct bug in computing latitude band X.

enhance test to cover exceptional width of band X.

update shapelib to 1.4.1 (#274)

enhance sort filter for rtes & trks. (#273)

use bool, true, false. (#272)

add msvc2017 builds to appveyor. (#270)

update docker build script to use latest bionic. (#269)

Merge pull request #266 from tsteven4/clazy1

Incorporate improvements found by Qt oriented code checker clazy

Merge pull request #267 from gpsbabel/gpsbabel-patch-2

Remove leftover code from 'adding class' change

Remove leftover code from 'adding class' change

As tsteven4 points out, No reasing to move is_internal to local and back

fix clazy detected 'use isEmpty() instead'

this is from -Wclazy-isempty-vs-count

fix clazy detected 'Use *Ref() instead'

These are from -Wclazy-qstring-ref, and were automatically fixed.

fix clazy detected 'unused *'

these are from -Wclazy-unused-non-trivial-variable

fix clazy detected 'allocating an unneeded temporary container'

these are from -Wclazy-container-anti-pattern

Also, consistently use range based for loops in csv_util.cc

fix clazy detected 'c++11 range-loop might detach Qt container'

these are from -Wclazy-range-loop

fix clazy detected 'Use multi-arg instead'

these are from -Wclazy-qstring-arg

update travis for xenial (#265)

* update travis to use xenial.

add new travis target to build on xenial.
switch travis coverage generation to xenial.
update codacy upload tool for compatibility with newer java on xenial.

* fix gcov issues with "cannot open notes file"

support kml 2.3 track element. (#264)

* support kml 2.3 track element.

relax ordering requirement for gx:Track when and
coord children.

* fix kml includes and indentation.

Merge pull request #261 from gpsbabel/gpsbabel-patch-2

Update dependencies, notably for <optional>

Replace queues with Qlist in session handling. (#263)

session tracks a possible series of input formats and input filenames.

Merge pull request #262 from tsteven4/tcache

extend travis mac cache timeout.

extend travis mac cache timeout.

Merge pull request #260 from tsteven4/cpp14_msvc

move to c++14 standard.

Merge pull request #259 from tsteven4/queue_bye_bye

eliminate queues in csv style handling.

Update dependencies, notably for <optional>

Merge pull request #256 from tsteven4/rpath

have configure conditionally set rpath.

move to c++14 standard.

eliminate queues in csv style handling.

also:
fix bug with style option encoding.
pass some parameters by const reference.
eliminate some unused time fiddling.

Merge pull request #258 from tsteven4/sortbug

correct sort filter time option to use msecs.

correct sort filter time option to use msecs.

use consistent whitespace in configure comments.

have configure conditionally set rpath.

this may be necessary when using a Qt installation
that was not provided by the system.

Merge pull request #255 from gpsbabel/parallel

Parallelize CLI and GUI make.

Parallelize CLI and GUI make. testo and vtesto are harder.

Merge pull request #254 from gpsbabel/tweaks

Move more xcsv processing into QStrings. Fewer conversions

Allow string to be modified during parse. Ugh.

Let xcsv parser keep QStrings up for one more level.

Push QStrings -> char * conversions down one more level.

Merge pull request #253 from gpsbabel/tweaks

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: G…

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: GNU sed is /usr/local/bin/

Merge pull request #252 from gpsbabel/tweaks

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: GNU sed is /usr/local/bin/

Merge pull request #249 from codacy-badger/codacy-badge

Add a Codacy badge to README.md

Add codacy badges to Github README.md

Add Codacy badge

Merge pull request #246 from tsteven4/trackdata

introduce optional class

introduce optional class

for things that may or may not have a value.
demonstrate usage with computed_trkdata.

Merge pull request #245 from tsteven4/exif2

a few exif cleanups.

a few exif cleanups.

Merge pull request #230 from tsteven4/bng

allow unicsv bng input to use all numeric grid references.

tweak unicsv bng documentation thanks Hornbydd.

Merge pull request #238 from tsteven4/exif

update exif format replacing queues, legacy time_t, tm, localtime, gmtime with Qt classes.
enhance exif rational number approximation using continued fractions.
support uncompressed thumbnails on write.
make debug support dynamic based on debug level instead of a conditional compile.
enhance exif test for write.

add exif support for uncompressed thumbnails.

fix exif dynamic debug omissions.

fix exif warnings.

replace legacy time handling in exif.

also add support of exif OffsetTime* tags.

minor fixes for exif.

use QVariant to store exif tag data.

enhance exif debug capability.

use continued fractions to create TIFF RATIONAL types.

some references were regenerated as the new algorithm
can generate different values for the RATIONAL numerator
and denominator, in this case 0.0 -> 0/INT32_MAX.

improvements to exif format.

fix bugs with rational types including greatest common divisor.
turn off assertions for release builds in GPSBabel.pro.
clean up exif.

tweak endian routines for consistency. (#242)

align util routeines with prototypes using readability-inconsistent-declaration-parameter-name.

eliminate queues in inifile. (#239)

* eliminate queues in inifile replacing them with QHashs

* replace use of gbfile in inifile with QTextStream